Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updates to fix current bugs in the project, and adds an example #17

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

justinmichaelvieira
Copy link

Fixes an extra ')' in master, as well as fixes for Example 1 not returning more than 5 results.

Also added an example shows cycling through all results, and adding a random delay to prevent google from blocking your requests.

tomMulholland and others added 4 commits April 22, 2014 20:36
modified the functions _extract_results and _extract_description according to these suggestions: http://www.catonmat.net/c/27124
…cause it would not return any results missing a Title or Desc - most were missing Desc - apparently this may have become way more common recently, or there may be a bug retrieving it somewhere?

Changed this so it now returns the correct 50 results, ignoring the incomplete fields.
…od citizen" to Google by waiting a bit after each results page.

Also turned on "random_agent" user-agent randomization by default.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants